Skip to content

Update permissions in cd.yaml#55

Merged
dai-shi merged 1 commit intomainfrom
dai-shi-patch-1
Dec 14, 2025
Merged

Update permissions in cd.yaml#55
dai-shi merged 1 commit intomainfrom
dai-shi-patch-1

Conversation

@dai-shi
Copy link
Member

@dai-shi dai-shi commented Dec 14, 2025

Add permissions for id-token and contents in CD workflow

Forgot in #52

Add permissions for id-token and contents in CD workflow
@dai-shi dai-shi merged commit 1352250 into main Dec 14, 2025
4 checks passed
@dai-shi dai-shi deleted the dai-shi-patch-1 branch December 14, 2025 04:31
@dai-shi dai-shi requested a review from Copilot December 14, 2025 05:12
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds explicit permissions configuration to the CD workflow file to properly scope the workflow's access rights. The permissions were previously missing and needed to be added as a follow-up to PR #52.

Key Changes:

  • Added permissions block with id-token: write and contents: read to the CD workflow

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants